Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feat/giveaway blacklist #137

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Feat/giveaway blacklist #137

wants to merge 15 commits into from

Conversation

hilmoo
Copy link
Member

@hilmoo hilmoo commented Jul 21, 2024

  • testing

@hilmoo hilmoo requested a review from Iqrar99 as a code owner July 21, 2024 16:05
@hilmoo
Copy link
Member Author

hilmoo commented Jul 21, 2024

this pull req extend #136

@hilmoo
Copy link
Member Author

hilmoo commented Jul 22, 2024

nvm, i forgot carl bot can handle that

@hilmoo
Copy link
Member Author

hilmoo commented Aug 3, 2024

waiting for #136 to merge for testing

bot/data/db.sql Outdated Show resolved Hide resolved
Comment on lines 28 to 29
winner='Winner of the giveaway. Example: 1234567890,0987654321',
ghosting='Member who do not claim the giveaway. Example: 1234567890,0987654321',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The example is too long for a parameter description

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think it's too long, it helps users understand how to separate them if more than one is entered

bot/cogs/giveaway.py Outdated Show resolved Hide resolved
bot/cogs/giveaway.py Outdated Show resolved Hide resolved
bot/cogs/giveaway.py Outdated Show resolved Hide resolved
bot/cogs/giveaway.py Outdated Show resolved Hide resolved
bot/cogs/giveaway.py Outdated Show resolved Hide resolved
bot/cogs/giveaway.py Outdated Show resolved Hide resolved
bot/cogs/giveaway.py Outdated Show resolved Hide resolved
bot/cogs/giveaway.py Outdated Show resolved Hide resolved
@hilmoo
Copy link
Member Author

hilmoo commented Sep 21, 2024

The code still has a syntax error. I’ll check it later, but for now, I can't test it on Discord.

@hilmoo
Copy link
Member Author

hilmoo commented Sep 21, 2024

already tested the feature. lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants